Skip to content

Conversation

@vlasenkoalexey
Copy link
Contributor

No description provided.

@vlasenkoalexey vlasenkoalexey requested a review from MarkDaoust May 12, 2020 19:39
Copy link
Member

@terrytangyuan terrytangyuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of being an user-level tutorial, should this be a developer guide that's linked in https://github.com/tensorflow/io/blob/master/CONTRIBUTING.md?

@yongtang
Copy link
Member

@terrytangyuan I think the content is suitable for part of the https://tensorflow.org/io, so the tutorial format might be good for developers to follow.

@vlasenkoalexey
Copy link
Contributor Author

@yongtang is right, I'm following existing tf.io docs structure to make sure that tensorflow docs integration works as expected. Will be happy to move / restructure if necessary.

@yongtang
Copy link
Member

/cc @tensorflow/docs-team to take a look.

@yongtang yongtang requested a review from lamberta May 13, 2020 16:40
@lamberta
Copy link
Member

Seems out of place for tutorials: https://www.tensorflow.org/io/overview ---which I expect to contain example of tasks that use TFIO.

But projects can have install and build guides, for example: https://www.tensorflow.org/federated/install
Quickly looking through and this looks like a build guide, yes? (I.e. not just how to use VS Code)

@yongtang
Copy link
Member

@lamberta @vlasenkoalexey I think have a page of install like https://www.tensorflow.org/federated/install would make sense. Do you know what will need to be updated or change in the PR?

Should the content be placed in a directory called docs/install instead (was in docs/tutorial)?

@lamberta
Copy link
Member

Should the content be placed in a directory called docs/install instead (was in docs/tutorial)

Yeah, I'd move it out of tutorials to the /docs root. Probably don't need a docs/install directory until there's more content to put there.

A lot of projects have something similiar, here's the TFQ install: https://www.tensorflow.org/quantum/install (and here's how their _book.yaml) is set up)

@vlasenkoalexey
Copy link
Contributor Author

Moved vscode instructions to docs root, @lamberta please take a look

@lamberta
Copy link
Member

Thanks.

Need to add to the leftnav somewhere. Not sure how this project is set up, I see a docs/_toc.yaml file and a docs/tutorials/_toc.yaml file---I assume this should be added to the former cc: @MarkDaoust

@MarkDaoust
Copy link
Member

Yes, docs/_toc.yaml makes the first section, tutorials/_toc.yaml makes the second section in the left-nav here:

https://www.tensorflow.org/io/overview

@vlasenkoalexey
Copy link
Contributor Author

As far as I understand it should go below Overview section in docs/_toc.yaml similarly to
https://www.tensorflow.org/quantum/install
Please take a look

Copy link
Member

@lamberta lamberta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me ..
last thing, can you create a docs/images directory and move the image in there. Thanks

@vlasenkoalexey vlasenkoalexey requested a review from lamberta May 26, 2020 18:27
@yongtang yongtang merged commit 5a64b0c into tensorflow:master May 26, 2020
i-ony pushed a commit to i-ony/io that referenced this pull request Feb 8, 2021
* Added Visual Studio Code tutorial, and updated README

* merged and reformatted

* added missing line break

* moved vscode setup instructions to docs root

* added VS code instructions to leftnav below Overview section

* moved docs/vscode/vscode_debugger.png -> docs/images/vscode_debugger.png
zheolong pushed a commit to zheolong/io-1 that referenced this pull request Jul 24, 2025
* Added Visual Studio Code tutorial, and updated README

* merged and reformatted

* added missing line break

* moved vscode setup instructions to docs root

* added VS code instructions to leftnav below Overview section

* moved docs/vscode/vscode_debugger.png -> docs/images/vscode_debugger.png
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants